Skip to content

Authorization#11

Open
evans22j wants to merge 24 commits intodevfrom
authorization
Open

Authorization#11
evans22j wants to merge 24 commits intodevfrom
authorization

Conversation

@evans22j
Copy link
Owner

@evans22j evans22j commented Aug 4, 2022

In this PR

  • Install CanCanCan in the project.
  • Add a role column to the users table. Remember to use a migration for this.
  • A user can delete a post if it is theirs or if they have an admin role (column role has value "admin"). Use CanCanCan for this authorization.
    • For that implement the post deleting functionality. Add the "Delete" button to the view and make sure that
      only authorized users can see it.
  • A user can delete a comment if it is theirs or if they have an admin role (column role has value "admin").
    Use CanCanCan for this authorization.
    • For that implement the comment deleting functionality. Add the "Delete" button to the view and make sure that only authorized users can see it.

evans22j added 24 commits August 2, 2022 16:04
@evans22j
add devise gem configuration
a14160a
@evans22j
update styles
c4c3ab2
@evans22j
add user devise config in application controller
26aca8b
@evans22j
create new html file in devise folder
50e2873
@evans22j
Update new.html.erb in registrations and sessions
78ea350
@evans22j
Update application.erb in views/layouts
79d7be3
@evans22j
Run rails db:migrate, and update index and shot html file in views/users
3ee66aa
@evans22j
add confirmable
25cd143
@evans22j
add devise gem configuration
5f420cf
@evans22j
Run rails db:migrate, and update index and shot html file in views/users
5c29be8
@evans22j
add user devise config in application controller
52ffcd4
@evans22j
resolve linter errors
9e74141
@evans22j
Update comments_controller.rb and create destroy method
5820d15
@evans22j
Update post_controller.rb and create destroy method
1612884
@evans22j
add cancancan authorization
8670658
@evans22j
Update comment.rb and create update_comments_counter_on_destroy method
e58a588
@evans22j
Update post.rb
b468531
@evans22j
Update user.rb
2779baf
@evans22j
Update posts show.html.erb to include delete option
fba2715
@evans22j
Update users show.html.erb
d0a59b3
@evans22j
Update routes.rb to include destroy action
fe51071
@evans22j
add cancancan authorization
a8cb60b
@evans22j
add cancancan authorization
74ec27d
@evans22j
correct linter errors
68ff6cf
zainsadaqat
zainsadaqat approved these changes Aug 4, 2022
View reviewed changes
Copy link

@zainsadaqat zainsadaqat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

APPROVED!!!! 🎉 🎉 🎉 🎉 🎉 🎉

Hey @evans22j 😊 😃

Great Job 😄 👏 You've done it. Your project is complete! There is nothing else to say other than... it's time to merge it 👍🏻 ✅

Congratulations!!! 🎉 🥳 👏 🙌 🎊

Cheers and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@zainsadaqat zainsadaqat zainsadaqat approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@evans22j @zainsadaqat